Bring brink skip_allocate

This was removed by accident in 580aa8204f45768,
together with some no longer needed popover
special-casing.

docs: Remove mention of queue_draw_area()

and queue_draw_region(). They're gone since 73650c6da2b6171d10633038f74088980e8a7fc7.

rendernodeimpl: Inline container_node_get_bounds into _new

It's the only caller of the function and we can avoid a second loop over
all child nodes this way.

snapshot: Make append_node_internal transfer-full

There is only one case where we do *not* directly unref the node again
after calling this.

Merge branch 'for-master' into 'master'

For master

See merge request GNOME/gtk!911

widget-factory: Show proper separators

builder: Allow values of type GType

In particular this allows setting the item-type property of GListStore.

builder: Make GListStore buildable

Treats GListStore like any other object and allows <child> to add items
to it.

testsuite: Add tests for printing/parsing transforms

Merge branch 'transform-tests' into 'master'

Transform tests

See merge request GNOME/gtk!910

gsk: Add some more transform tests

Tests around identity. Seems trivial, but
still managed to find a bug.

gsk: Fix printing of identity transforms

These need to print as "none", otherwise
the parser does not accept them.

Cosmetics

Fix up a variable name mismatch in GskTransform docs.

Move transform tests to the gsk suite

That is where they belong.

Merge branch 'for-master' into 'master'

rendernodeparser: Handle Cairo being stupid

See merge request GNOME/gtk!909

GskTransform: NULL is a valid transform

rendernodeparser: Handle Cairo being stupid

Cairo writes to finished streams, so we have to make sure to keep the
stream around to allow it to do that.

gl renderer: Dont' memcmp() a GskTransform

widget: Unref priv->{allocated_,}transform

gl renderer: Don't ref transform twice

That's gonna leak it otherwise.

glyphcache: Don't leak hashtables

GskTransform: Ignore identity transforms

We often end up with e.g. a scale of 1 or a translation by 0/0. Ignore
those transforms since they don't do anything.

gl renderer: Ref transform before using it

Fixes #1929

gl renderer: Dont' unnecessarily to_matrix transforms

Merge branch 'menu-sizing' into 'master'

menu: Update scrollbar policy

Closes #1930

See merge request GNOME/gtk!908

menu: Update scrollbar policy

We want to avoid sizing problems for small menus
due to the vertical scrollbar imposing a min-height on
its slider.

Closes: https://gitlab.gnome.org/GNOME/gtk/issues/1930

Merge branch 'cherry-pick-ae2ef147' into 'master'

gtk: fix weed_out_neg_zero() in spinbutton and scale

See merge request GNOME/gtk!906

Updated Spanish translation

Merge branch 'glyph-memdup' into 'master'

gl: Avoid a memdup in glyph upload

See merge request GNOME/gtk!907

gl: Avoid a memdup in glyph upload

We don't need to dup the memory here if we
set up the image surface properly. This won't
matter for most glyphs, but some of them can
be big.

Merge branch 'big-glyphs' into 'master'

Big glyphs

See merge request GNOME/gtk!904

Save gsk output from ci

testsuite: Pass --output for gsk tests

We want to keep the output from ci runs.
So we now write the .out.png and .diff.png
files to build/testsuite/gsk/compare/$RENDERER.

gsk tests: Add an --output option

This will be used to save artifacts in ci.

gl: Fix glyph cache aging

The logic here seems faulty. We want to keep
a timestamp that tells us when the glyph was
last used, so always update the timestamp.

And whenever we use a glyph, it turns 'young'
again, so remove it from the old pixels
accounting.

The (MAX_AGE, MAX_AGE+CHECK_INTERVAL) interval
is only relevant to prevent us from turning
a cached glyph old more than once, and that
is already taken care of.

Add code to dump out the glyph caches

gl: Improve cache dropping code

There was a TODO here to avoid iterating the
glyphs multiple times, so avoid that. And
actually log the number of glyphs that was dropped.

gl: Improve glyph cache logging

Use g_message to be consistent, and make the
message fit a line.

gl: Log creation of glyph caches

Now that they can be of different sizes, this
is somewhat interesting.

gsk: Test huge glyphs

These don't fit in the glyph cache.

gl: Make the glyph cache survive big glyphs

Create an extra atlas of just the right size for
each huge glyph. Not pretty, but works.

gsk: Test big glyphs

These are big, but still fit in the glyph cache.
Therefore, the test succeeds.

gtk: fix weed_out_neg_zero() in spinbutton and scale

It was not copying the terminating \0 in the string, breaking output
in spinbutton (didn't try scale). Fixes #3452.

(cherry picked from commit ae2ef1472c69ad61ed8d19d932da4ca04c2a13ef)

Merge branch 'for-master' into 'master'

For master

See merge request GNOME/gtk!903

Merge branch 'wip/chergert/marshalling-gtk4' into 'master'

gtk: fix all uses of g_cclosure_marshal_generic

See merge request GNOME/gtk!905

gtk: fix all uses of g_cclosure_marshal_generic

This adds specific marshallers for all of the locations where a generic
marshaller is being used. It also provides va_marshallers to reduce the
chances that we get stack traces from perf going through ffi_call_unix64.

This is forward ported from gtk-3-24.

# Conflicts:
# gtk/gtkeventcontrollerkey.c
# gtk/gtkeventcontrollermotion.c
# gtk/gtkgesture.c
# gtk/gtkgesturemultipress.c

gtk-demo: Fix sliding puzzle demo

1. Menubuttons aren't buttons

2. For paintables without aspect ratios, use 1.0

transform: Coalesce similar transforms

If somebody does a transform like
  scale(5) scale(10) translate(1,1) translate(5,0)
store it instead as
  scale(50) translate(6,1)
This way, less memory is consumed and transforms are easier to read.

In particular, this simplifies the typical transforms we do in GTK,
which are just one translation after another.

Don't generate mipmaps needlessly

GL_LINEAR doesn't use mipmaps, so don't generate them.

transform: Treat 0 perspective transforms as identity

This happens if you apply a perspective transform and its inverse, which
is the negative version of the perspective.

Update Polish translation

gl renderer: Remove homegrown transform_bounds code

GskTransform can do this for us now.

transform: Add G_GNUC_WARN_UNUSED_RESULT annotations

These functions return the new transform so their return value should
really never be ignored.

gl renderer: Fix scale computation

We don't need to just look at the scale of the new modelview matrix, but
at the one we get when multiplying the new one with the current one.

Test case attached.

gl renderer: Use GskTransform in render ops builder

for the modelview matrix. We need this later.

gl renderer: Fix rounded rect intersection code

Test case attached.

Fixes #1920

Add gtkmenubarprivate.h

Move private api into a private header.

Cosmetics

Remove comments that are old enough to still
talk about option menus and item factories.

Merge branch 'restack-gtk4' into 'master'

GDK W32: Ensure correct stacking of popup windows (GTK4)

See merge request GNOME/gtk!902

GDK W32: Ensure correct stacking of popup windows

1) In the SetWindowPos() function (and the WINDOWPOS struct) the
   "hWndInsertAfter" argument/field means the window that will be
   directly above after the change, not the window that will be
   directly below. MSDN says "precedes" for SetWindowPos(), but
   WINDOWPOS documentation is more precise: this is the window
   behind which the affected window will be placed. Apparently,
   Z-axis goes back-to-front.
   Therefore, logging should be reworded correctly.

2) When we switch away from the application and then switch back
   to a transient window, we need to bring up its transient-owner
   (and its transient-owner's owner and so forth) as well,
   otherwise our transient (modal) window might be transient for
   something that might not be visible.

3) When we bring up a window, we should bring all of its children
   (popup windows) on top of it.
   Because Windows doesn't provide a function to bring one window
   on top of the other, we have to work around this by calling
   SetWindowPos() twice, swapping the windows between the calls.

Update Hungarian translation

Remove an unused enum

No use of GtkArrowPlacement anywhere.

Merge branch 'global-coords' into 'master'

Drop global coordinates

See merge request GNOME/gtk!899

Update css node results

Fix menu item accessible

The menu shell is not a direct parent anymore.

tests: Fix issues with menu shells

The menu shell is no longer the direct
parent of menu items.

menu: Fix destruction

Now that menubar and menu are containers
with internal structure, we need to be
careful about doing the right thing in
forall and dispose.

Merge branch 'convert-dialog' into 'master'

Convert dialog

See merge request GNOME/gtk!898

menu: Fix keynav between submenus

Left/Right arrow now work again to enter
or leave a submenu.

menu: Bring back the scroll keybindings

Home, End, Page Up/Down, work again.

menu: Keep selected item in view

This gets us most of the way to working scrolling.

More menu work

More menu restructuring

Make GtkMenuBar use a box as well,
and let GtkMenuShell get the items
from GtkMenuBar and GtkMenu via
a vfunc. Use that to fix the keynav
implementation in GtkMenuShell.

menu: Use a scrolled window

This brings back some support for scrolling.
We still need to reinstate some of the
scroll-to-selected and keynav functionality.

menu: Use a box

This makes some of the GtkMenuShell api
no longer work, since we don't let the
menu shell code maintain the list of
children anymore.

Drop gdk_event_get_root_coords

widget: Stop setting root coords in events

Nothing should use them anymore.

win32: Stop using gdk_event_get_root_coords

It seems we want local coords here anyway.

menu: Remove global coordinates

This commit removes support for scrolling and
for the keep-up triangle from GtkMenu, and gets
rid of all use of global coordinates.

menu: Only set position in one place

menu: Simplify function

No need to do complicated math when we can just look at the allocation.

menuitem: Introduce gtk_menu_item_get_menu_shell()

Returns the parent menu shell or NULL.

Replace all calls to gtk_widget_get_parent() with this function.

simplify: Add a testcase for templates

simplify: Add a testcase for dialogs

Update POTFILES.in

builder-tool: Rewrite GtkDialog

Changes

  <object class="GtkDialog">
    <child internal-child="vbox">
      <...>
        <child internal-child="action_area">...</child>
        ...
      </...>
      <packing />
    </child>
  <object>

to

  <object class="GtkDialog">
    <child internal-child="content_area">
      <...>
        ...
      </...>
    </child>
    <child internal-child="action_area">...</child>
  <object>

Merge branch 'overlay-scrolling-setting' into 'master'

Overlay scrolling setting

See merge request GNOME/gtk!873

scrolled window: respect overlay-scrolling setting

If the gtk-overlay-scrolling setting is FALSE,
don't use overlay scrollbars.

Add a gtk-overlay-scrolling setting

This is in preparation for letting user opt out of
overlay scrolling in the control-center.

Merge branch 'wip/chergert/remove-cclosure' into 'master'

gtk: rely on default marshallers

See merge request GNOME/gtk!893

Merge branch 'for-master' into 'master'

For master

See merge request GNOME/gtk!892

gtk: rely on default marshallers

Similar to previous removals of g_cclosure_marshal_VOID__VOID we can remove
other marshallers for which are a simple G_TYPE_NONE with single parameter.
In those cases, GLib will setup both a c_marshaller and va_marshaller for
us. Before this commit, we would not get a va_marshaller because the
c_marshaller is set.

Related to GNOME/Initiatives#10

widget: fix CSS transforms with margins

The CSS transform should operate on the border-box, not the margin box.
So we need to shrink the bounds by the margin before we apply the CSS
transform.

gtk-demo: Fix sliding puzzle demo

1. Menubuttons aren't buttons

2. For paintables without aspect ratios, use 1.0

Merge branch 'wip/nielsdg/fix-gir-warnings' into 'master'

Fix some GObject introspection annotations

See merge request GNOME/gtk!891

Fix some GObject introspection annotations

Fixes several warnings by the GIR compiler.

Add a testcase

This tests the fix in 1e7225aa1c20525da0.

build: Make script interpreter optional

Running the tests needs it though, so it's only optional without tests.

docs: Fix build

Merge branch 'cairo-script-interpreter' into 'master'

rendernodeparser: Parse cairo script

See merge request GNOME/gtk!876